Record media integrity in the State table - #48
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the project’s high-level “State” table to include a concise, durable summary of the media-integrity outcome so a new reader can find the conclusion in the central status summary.
Changes:
- Add a new “Media integrity” row to the
Statetable inTODO.md. - Document the defended
ORPHANED_MEDIAinvariant and the adjudicated orphaned-media count.
ptr727
force-pushed
the
record-media-integrity-state
branch
from
August 8, 2026 13:06
59a4cae to
071f58c
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
TODO.md:12
- The phrase "the 98 files no page links" is grammatically incomplete and reads like a missing preposition, which makes the row harder to parse. Consider changing it to "the 98 files with no page links".
| Media integrity | closed for the files themselves. The conversion's only image loss was 19, inside five galleries, restored from the capture, and the 98 files no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |
ptr727
force-pushed
the
record-media-integrity-state
branch
from
August 8, 2026 13:12
071f58c to
f16096e
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
TODO.md:12
- The phrase “the 98 files no page links” is grammatically incorrect/missing a word; it reads like “with no page links”. This affects readability of the State table row.
| Media integrity | closed for the files themselves. The conversion's only image loss was 19, inside five galleries, restored from the capture, and the 98 files with no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
TODO.md:12
- This sentence uses a bare number ("loss was 19") which can be read as files/bytes/etc. Consider explicitly stating "19 images" (matching the PR description) to make the unit unambiguous in this summary table.
| Media integrity | closed for the files themselves. The conversion's only image loss was 19, inside five galleries, restored from the capture, and the 98 files with no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |
The table said nothing about whether the conversion lost anything, which is the one question a migration's record has to answer and the one a reader cannot check for themselves. Rebased onto develop and reconciled with two things that landed while this sat open. The volatile counts it originally carried in the Content and media row are gone, because develop deliberately replaced counts that content additions move with what they represent, and this row would have reintroduced 514 pages against that decision. And the claim is narrowed from "closed" to closed for the files themselves: the image loss was 19 and is restored, but #55 later found the conversion had also flattened how galleries present media, in three shapes across three posts. That was not a file loss and no file was missing, which is exactly why nothing saw it for so long, and it is now gated. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ptr727
force-pushed
the
record-media-integrity-state
branch
from
August 8, 2026 13:15
f16096e to
4cde232
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
TODO.md:12
- The added table row’s wording doesn’t match the PR description’s proposed "Media integrity" State-table entry (it adds extra qualifiers and an additional sentence about presentation/gating). Either update the PR description or adjust the row text so the State table remains the single-source summary as described.
| Media integrity | closed for the files themselves. The conversion's only loss was 19 images, inside five galleries, restored from the capture, and the 98 files with no page links are adjudicated rather than unknown, with `ORPHANED_MEDIA` failing if that count moves either way. How media was *presented* was a separate loss, found later and now gated |
Owner
Author
|
Answering the suppressed finding, which carried no thread.
Correct, and the description was the stale half. This branch was rebased onto Taking the first of your two options: the description now quotes the row verbatim and explains both changes to it.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
One row. The
Statetable answers "where is this" for someone arriving cold, and it had no row for the surface that took the most work to establish. A reader would have found the media question discussed inchecks/README.md, inOPERATIONS.md, and in this file's own sections, and unanswered in the one place that summarizes.Rebased onto
developand reconciled, having sat open since 2026-08-07. The row is not the one this branch originally proposed, for two reasons.The claim is narrowed from "closed" to "closed for the files themselves." #55 later found that the conversion had also flattened how galleries present media, in three shapes across three posts: eleven set captions written as trailing text, two bare markdown images, and eight linked ones. None of that was a file loss and no file was missing, which is exactly why nothing saw it — every media check reasons about a URL, and misplaced content resolves and is linked just as correct content does. Saying "closed" flat would have been true of the files and misleading about the surface.
The
Content and mediarow is left alone. This branch originally rewrote it to carry "514 pages, 778 media files".develophas since deliberately replaced counts that content additions move with what they represent, keeping only the counts that describe the legacy contract. Reintroducing a page count would have reverted that decision, so the conflict is resolved in develop's favour and only the new row is added.The row states the conclusion rather than the activity, and names
ORPHANED_MEDIAbecause the useful part for a later reader is that the number is defended in both directions now, not that it was once measured.Verification
markdownlint-cli2andeditorconfig-checkerclean. Documentation only — no code, config, workflow, or content change.🤖 Generated with Claude Code